Skip to content

feat(cli,resources): pf-4059 catch start issues#179

Merged
cdcabrera merged 2 commits intopatternfly:mainfrom
cdcabrera:pf-4059
May 4, 2026
Merged

feat(cli,resources): pf-4059 catch start issues#179
cdcabrera merged 2 commits intopatternfly:mainfrom
cdcabrera:pf-4059

Conversation

@cdcabrera
Copy link
Copy Markdown
Member

@cdcabrera cdcabrera commented May 1, 2026 •
edited
Loading

What is it?

  • feat(cli,resources): pf-4059 catch start issues
    • docs, troubleshooting index
    • agent, agnostic checking for node versioning, env checks using resources
    • build, custom support url to package
    • cli, expose troubleshooting links, catch 3 additional levels of errors, engine, package load, runtime
    • options, move getNodeMajorVersion to helpers
    • resources, patternFlyContext, expose env and troubleshooting links

Notes

  • adds in a specific CLI e2e to confirm failures
  • a funky minor cursor output error popped up during debugging. their debugger/logger pops \sundefined onto the end of messaging. we confirmed its not the PF MCP output

Check the work

  1. Update the NPM packages with $ npm install
  2. $ npm run build
  3. npx -y @modelcontextprotocol/inspector node dist/cli.js
  4. connect and list resources
  5. confirm the patternfly://context resource contains environment listings

Check the build

  1. Update the NPM packages with $ npm install
  2. $ npm run build
  3. confirm the checks come back clean

Updates issue/story

pf-4059
closes #172

@cdcabrera cdcabrera marked this pull request as draft May 1, 2026 20:58
@cdcabrera cdcabrera changed the title feat(cli,resources): pf-4059 catch load, start issues feat(cli,resources): pf-4059 catch start issues May 1, 2026
@cdcabrera
feat(cli,resources): pf-4059 catch cli start issues
e6aa695 
* docs, troubleshooting index, agent troubleshooting guidance
* agent, allow checking for node versioning via mcp resource
* build, add custom support url to package.json
* cli, expose 3 levels of error with troubleshooting links
* options, move getNodeMajorVersion to helpers, troubleshooting instructions
* resources, patternFlyContext, expose env and support links
@cdcabrera cdcabrera marked this pull request as ready for review May 4, 2026 16:17
@patternfly-build
Copy link
Copy Markdown

patternfly-build commented May 4, 2026 •
edited
Loading

✅ Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
✅ Open Source Security 0 0 0 0 0 issues
✅ Licenses 0 0 0 0 0 issues
✅ Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@cdcabrera cdcabrera merged commit 5974f60 into patternfly:main May 4, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add a troubleshooting solution for recent MCP server updates

2 participants

@cdcabrera @patternfly-build